From 2a4c343c5baa241ea5d8685ac96989543ea67410 Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Fri, 31 May 2019 11:39:49 +0200 Subject: [PATCH] VT-d: change bogus return value of intel_iommu_lookup_page() The function passes 0 as "alloc" argument to addr_to_dma_page_maddr(), so -ENOMEM simply makes no sense (and its use was probably simply a copy-and-paste effect originating at intel_iommu_map_page()). Signed-off-by: Jan Beulich Reviewed-by: Paul Durrant Acked-by: Kevin Tian --- xen/drivers/passthrough/vtd/iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/drivers/passthrough/vtd/iommu.c b/xen/drivers/passthrough/vtd/iommu.c index 7b9e09a084..4caf63c7fa 100644 --- a/xen/drivers/passthrough/vtd/iommu.c +++ b/xen/drivers/passthrough/vtd/iommu.c @@ -1870,7 +1870,7 @@ static int intel_iommu_lookup_page(struct domain *d, dfn_t dfn, mfn_t *mfn, if ( !pg_maddr ) { spin_unlock(&hd->arch.mapping_lock); - return -ENOMEM; + return -ENOENT; } page = map_vtd_domain_page(pg_maddr); -- 2.30.2